Port threads to Django syntax #1432
Conversation
|
@brylie you can take a look at this screen record to gain insight into the final design:
|
There was a problem hiding this comment.
Overall, this PR looks good. Let's just attach the related Civi queries for problems, causes, and solutions directly to the Thread class.
Let's also discuss the performance implications of civi/solution counts on the Thread model and where/whether those counts will be used.
Let's use #1152 as the parent issue for the pull request that entirely removes Backbone.js.
If we add those issues with good descriptions, we can label them with |
Related to #1152
I will create new issues to fix creation of new civis, voting, adding response and recommended civis. I feel like there are some leftovers. And we need to remove thread.js. file. I am planning to create another issue (a similar one to #1151) to clean up.